West Midlands | ITP - January 2026 | Jonathan Boahene | Sprint 2 | Form Control#1193
West Midlands | ITP - January 2026 | Jonathan Boahene | Sprint 2 | Form Control#1193Abayie wants to merge 4 commits intoCodeYourFuture:mainfrom
Conversation
Updated the HTML form to include customer information fields, color selection, size selection, and delivery date. Changed the footer to reflect the author's name.
✅ Deploy Preview for cyf-onboarding-module ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
cjyuan
left a comment
There was a problem hiding this comment.
-
Lighthouse Accessibility score not yet 100.
-
According to https://validator.w3.org/, there are several warnings in your HTML code. Can you address these warnings?
| type="text" | ||
| id="name" | ||
| name="user-name" | ||
| pattern="[A-Za-z\s]{2,}" |
There was a problem hiding this comment.
Do you know what this regular expression mean?
There was a problem hiding this comment.
yes I do know what regular expression mean
regular expression is a rule that describes what kind of text you’re looking for
There was a problem hiding this comment.
I mean, this particular one [A-Za-z\s]{2,}.
It does not quite match the description of the title attribute value:
Please enter your full name with at least two letters.
There was a problem hiding this comment.
I don't understand the question, and i don't see the challenge you're trying to address here
There was a problem hiding this comment.
What names does the pattern [A-Za-z\s]{2,} match?
The value of the title attribute is "Please enter your full name with at least two letters.". Does the "letters" include space characters?
There was a problem hiding this comment.
I know what you mean now, I mean in the case there are two ways to fix it, either change the regex to enforce full name as in the description or change the description from full name to just name
I will do the later as full name isn't necessarily required but want to enforce characters to the minimum
There was a problem hiding this comment.
To only enforce minimum number of characters in the input, there is a HTML attribute for that purpose.
Regex is more suitable if we need to ensure the input matches certain pattern. You can explore regex later.
fix accessibilty score and w3c validator warnings
|
All comments addressed and accessibilty score is 100% now |
|
There are still warnings in your HTML code -- it means some tags are not used properly. Can you fix them? |
|
Fixed the warnings |
|
Can you also address #1193 (comment) ? |
|
addressed comment #1193 |
|
Changes are good. Well done. |

Learners, PR Template
Self checklist
Changelist
Updated the HTML form to include customer information fields, color selection, size selection, and delivery date. Changed the footer to reflect the author's name.
Form control assignment
Questions
No questions